Skip to content

refactor: 검색 필터 파싱과 핀 요청 검증 책임#105

Merged
mike7643 merged 3 commits into
mainfrom
feat/locker-filter-validation-refactor
Jun 29, 2026
Merged

refactor: 검색 필터 파싱과 핀 요청 검증 책임#105
mike7643 merged 3 commits into
mainfrom
feat/locker-filter-validation-refactor

Conversation

@mike7643

@mike7643 mike7643 commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

✨ 작업 내용 한 줄 요약

  • 검색 필터 문자열 파싱을 웹 계층으로 옮기고 핀 조회 요청의 사용자 위치 검증을 DTO에서 벨리데이션

🛠️ 작업 내용

  • 검색, 장소별 조회, 핀 조회 요청이 문자열 집합 대신 애플리케이션 전용 enum 집합을 받도록 하여 타입 세이프 하게 변경
  • 스프링 Converter와 WebMvc 설정을 추가해 size, 실내외, 보관함 타입 파라미터를 요청 단계에서 정규화 및 변환하도록 변경
  • LockerSearchFilter 내부 문자열 파싱 책임을 제거하고 애플리케이션 계층이 타입 안전한 필터만 다루도록함
  • 핀 조회 요청에서 userLat, userLng 파라미터명을 명확히 맞추고 사용자 위치 누락 시 검증 오류가 발생

🧠 기술적 의사결정

  • 필터 문자열 검증과 포맷 보정은 도메인이나 애플리케이션이 아니라 컨트롤러 진입 이전에 끝내도록 분리

@mike7643 mike7643 requested a review from buddle031 as a code owner June 29, 2026 07:30
@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown

Caution

Review failed

An error occurred during the review process. Please try again later.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/locker-filter-validation-refactor

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@mike7643 mike7643 changed the title refactor: 검색 필터 파싱과 핀 요청 검증 책임 정리 refactor: 검색 필터 파싱과 핀 요청 검증 책임 Jun 29, 2026
@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown

Caution

Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted.

Error details
{}

@mike7643 mike7643 merged commit b4249e5 into main Jun 29, 2026
4 checks passed
@mike7643 mike7643 self-assigned this Jun 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant